fix(docs): add FastHTTP Postgres sample run and test instructions #766
Conversation
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
|
@Achanandhi-M sir, please take a look whenever possible. |
Achanandhi-M
left a comment
Achanandhi-M
left a comment
There was a problem hiding this comment.
Hey @Srinu346, please take a look at the other quickstarts to understand how they’re structured. What you’ve added currently feels more like a README, whereas the docs follow a different format. Take your time to review how we’ve added other quickstarts and align this with our existing guides.
Also, please include a video demonstrating your changes and add relevant images as well. Make sure the images have a white background.
|
@Achanandhi-M sir i will refine it and update it. |
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Screen.Recording.2026-01-30.001941.mp4@Achanandhi-M sir, please take a look when possible. |
|
Hi Sir, For reference, these are the images being used:
Please let me know if this is okay, or if you would prefer me to update/replace them with new screenshots. |
Achanandhi-M
left a comment
Achanandhi-M
left a comment
There was a problem hiding this comment.
Hey @Srinu346, please avoid removing the existing image from the file—don’t change anything there and use it as is. You can include additional images in the directory if needed; that’s not an issue.
Also, please include a short video showcasing your changes. One more request: kindly avoid using AI-generated text, as it appears that may have been used, and please remove emojis as well. Thanks for understanding.
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
…/docs into fix/quickstart-settings
|
@Achanandhi-M sir, Regarding emojis, I have removed most of the unnecessary ones and retained only those that are already used in other quickstart samples for consistency. I have also revised the content to remove any AI-generated text and updated it accordingly. Please let me know if any further changes are required. |
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
|
@Achanandhi-M sir here is a short video of demonstration: Screen.Recording.2026-02-02.230840.mp4 |
|
@Achanandhi-M Sir, i have only left out the emojis which were also being used in other setups |
Achanandhi-M
left a comment
Achanandhi-M
left a comment
There was a problem hiding this comment.
The changes look good, @Srinu346 . However, please don’t modify anything in the existing setup—only include the required updates. I noticed that some changes were made to the Docker Compose method; kindly avoid that. Also, I don’t see the images for the Keploy test that were mentioned in the previous review—please add those as well.
Signed-off-by: Akella Srinivas <alsrinivas.346@gmail.com>
@Achanandhi-M sir,
I have added the previously removed part back into the Docker Compose section. I’ve also included the above Keploy test image at the appropriate places both in the Docker Compose section and the WSL/Linux section. I haven’t removed or modified any of the existing images; only added the new ones where relevant. I’ve tried to limit the changes strictly to the sections where updates were required. |
|
@Achanandhi-M sir, here is a short video of demonstration. Recording.2026-02-03.183820.mp4 |
2 participants
What has changed?
This PR updates the FastHTTP + Postgres sample documentation to expose and document the existing Docker-based setup that was already implemented in the backend.
The changes include:
.mdfile to document both:This PR resolves #3269
Type of change
How Has This Been Tested?
Checklist: